跳到主要内容

2.3-code review

实际上,Code Review可以为我们带来大量的好处:

  • 提高代码质量:Code Review可以帮助开发者发现潜在的问题、漏洞和错误,以及提供更好的实现方式,从而提高代码质量。
  • 减少维护成本:在Code Review过程中,可以发现代码中的问题和漏洞,并及时修改,避免了在上线后出现问题需要大量时间和精力进行维护的情况。
  • 加强团队协作:Code Review可以促进团队内成员之间的交流和协作,提高团队整体的代码质量和开发效率。
  • 提高代码可读性:Code Review可以帮助开发者发现代码可读性的问题,例如命名、注释、代码结构等方面,从而提高代码的可读性。
  • 检查代码安全性:Code Review可以检查代码中的安全问题,例如XSS、CSRF、SQL注入等,从而提高代码的安全性。

意义

code review 更像是将统一的编程思想推广到整个团队,让所有人的思想保持一致

一开始 leader 一人发起,然后就可以交给团队内部进行自我 review,来了新人,也能保证代码团队内部一致

常见需要Code Review的问题点:

  1. 重要模块整体review
  2. 滥用全局变量
  3. 命名规范
  4. 闭包内部变量未被销毁
  5. 计时器是否及时清理
  6. 监听事件是否有解绑
  7. 第三方库的销毁函数,在页面卸载时也需要调用,比如EventBus:
  8. v-if 指令导致的内存泄露
  9. 异步操作是否有异常处理
  10. 组件是否需要拆分
  11. TypeScript中滥用any
  12. 未用防抖节流
  13. 动画减少回流与重绘
  14. 关键业务未加注释
  15. 避免重复注释
  16. 注释业务上的作用,而不是翻译一下英文名称